Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

白名单运行 #34

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

白名单运行 #34

wants to merge 2 commits into from

Conversation

RikaCelery
Copy link
Contributor

No description provided.

@RikaCelery
Copy link
Contributor Author

oops

RikaCelery
RikaCelery commented Feb 10, 2025
View reviewed changes
basic/ban/public.go
@@ -16,21 +16,29 @@ func SetUserPluginStatus(status bool, userID int64, plugin *manager.PluginCondit
var key string
if plugin != nil {
key = plugin.Key
} else { // plugin 为空，代表所有插件
key = AllPluginKey
Copy link
Contributor Author

@RikaCelery RikaCelery Feb 10, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

看了一下这里没有这种情况出现

PaimengBot/basic/ban/switch.go

Lines 53 to 58 in c8266f4

// 处理插件名
plugin = findPluginByName(args[0])
if plugin == nil {
ctx.Send(fmt.Sprintf("没有叫%v的功能哦，可以看看帮助", args[0]))
return 0, nil, period, fmt.Errorf("no such plugin %v", args[0])
}

RikaCelery
RikaCelery commented Feb 10, 2025
View reviewed changes
basic/ban/public.go
@@ -51,21 +59,29 @@ func SetGroupPluginStatus(status bool, groupID int64, plugin *manager.PluginCond
var key string
if plugin != nil {
key = plugin.Key
} else { // plugin 为空，代表所有插件
key = AllPluginKey
Copy link
Contributor Author

@RikaCelery RikaCelery Feb 10, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

同上，传入参数不可能为nil

PaimengBot/basic/ban/switch.go

Lines 53 to 58 in c8266f4

// 处理插件名
plugin = findPluginByName(args[0])
if plugin == nil {
ctx.Send(fmt.Sprintf("没有叫%v的功能哦，可以看看帮助", args[0]))
return 0, nil, period, fmt.Errorf("no such plugin %v", args[0])
}

@RikaCelery RikaCelery marked this pull request as draft February 10, 2025 02:52
@RikaCelery
fix: 全局白名单 群聊 黑名单 help显示不正确
e0875e2 
fix: 切换白名单后所有插件被禁用
@RikaCelery
Copy link
Contributor Author

等 #33 合了再弄，conflict太多了

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@RikaCelery